Notes shouldn't be part of this CL

I failed a wrestling match with Github

Dev Doc fix: Document Qt, Qmake better.

Better enumerate dependencies
Describe Qt Crerator
Improve internal linking
Fix trailing spaces

Remove remnants of NEW_STRINGS

NEW_STRINGS haven't been new for many years at this point. I'm tired of looking at that code. Most of what's left is in paths never executed or tested anyway. :-/

update qt to 5.12.5 (#403)

Merge pull request #400 from jspricke/manpage_xml

Fix manpage xml

Merge pull request #401 from spacelama/master

Don't dump garmin_fit from Lezyne "Super Fit" devices just because occasionally the endian field is not 0 or 1, leading to the question if that's an endianness field at all.

correct reference file for garmin_fit lezyne super gps - probably originally converted using wrong flags

The lezyne "super gps", every 10th ride or so, outputs a .fit file
that is entirely valid other than a weird non 0/1 endian field.  Don't
dump the data just because of that.

Fix manpage xml

fix lowranceusr4 writer issues (#376)

* fixes for lowraneusr writer for versions >= 4.

use fs_chain_find when looking for FS_LOWRANCEUSR4 data.
and check for nullptr to verify FS_LOWRANCEUSR4 data exists.

* using rounding in lowranceusr lat/lon conversions.

This gives as a chance of usr->gpx->usr matching.

Update gpsbabel generated usr reference files for these
rounding differences.

* fix lowranceusr4 writer issues.

when writing embedded UTF-16LE strings in lowranceusr version >=4
don't inlucde BOM.

don't offset returned values from lowranceusr4_jd_from_timestamp
by 12 hours, thus matching the inverse transformation in
lowranceusr4_get_timestamp.

when writing lowranceusr version >=4 trails output the given number
of mysterious attribute bytes, i.e. 0.

This almost enables a gpx -> usr -> gpx test loop, which is included
but commented in lowranceusr.test.

* fixes to enable lowranceusr v4 round trip testing.

use templates for lowranceusr*_find_desc_from_icon_number
and lowranceusr*_find_icon_number_from_desc.

enhance lowranceusr*_find_icon_number_from_desc to
recongnize icon-n where n is an integer as icon number n.
This is necessary for the v4 round trip test.

declare lowranceusr*_find_desc_from_icon_number to be static
to limit visibilty. Drop const return type.

declare lowranceusr4_find_color_from_icon_number_plus_color_index
to be static to limit visiblity.

* fix new lowranceusr memory leak.

fix minor error in garmin_fit lat/lon conversion. (#398)

Standardize resource access (#393)

* standardize resource access.

avoid search paths (use :/).
specify extensions.
ok/cancel only on windows.

* fix missing icon for file input in options dialog.

It appears the image was renamed when used on the main window UI
in the inputFileNameBrowseBtn.
The same icon, open.png, is now used on the options dialog for
OPTinFile.
This fixes #392.

fix interaction between translation and lights. (#390)

When using a language other than English the initial painting of
the gui mainwindow showed the placeholder text for the labels next
to the waypoints, routes and tracks check boxes instead of the
lights.  This was issue #389.

don't compile unwanted module serial_unix.cc on mac. (#388)

* don't compile serial_unix.cc on mac.

* remove unnecessary conditional in serial_unix.cc

File is no longer in SOURCES for mac, so we don't need to use a
conditional to skip the contents on mac.

Update README.contrib to reference GitHub (#383)

Merge branch 'master' of https://github.com/gpsbabel/gpsbabel

Fix a missed newline after Verion in debug code for Garmin Protocol.

Merge pull request #377 from jspricke/no_makeflags

Don't pass MAKEFLAGS to $(MAKE)

Fix Garmin debugging for "funny" characters. Isalnum() prints single byte characters that aren't ASCII, making funny business in iTerm. Yes, all those should be refactored into a single "print buffer".

Try to make the 'resettime' work per
https://github.com/gpsbabel/gpsbabel/issues/350
Debugging shows it's sending a time to my USB Garmin, but since this seems
to be a non-rollover device, the time is borked on mine anyway.

Don't pass MAKEFLAGS to $(MAKE)

make will pass them automatically. Also they are not meant to be
consumed by make as they are stripped:

Thus, if you do ‘make -ks’ then MAKEFLAGS gets the value ‘ks’.

https://www.gnu.org/software/make/manual/html_node/Options_002fRecursion.html

Merge pull request #374 from tsteven4/jeepswarn

Fix g++-8 warnings in jeeps.

Fix g++-8 warnings in jeeps.

modify position filter algorithm. (#373)

For routes and tracks, the algorithm is modified to
only remove consecutive points until the distance limit is
exceeded.  This prevents simulataneous removal of points on
an outgoing and return leg, which can lead to gross corruption
of the route/track shape.

A test case that demonstrates the problem with the previous
algorithm is added.

spelling fixes for GUI (#372)

* clion spelling fixes for GUI.

Care must be taken to not lose translations!

* fix clang tidy performance-unnecessary-value-param

Gdb string read (#364)

* harden gdb reader to long string input.

fix bug in the gdb reader that resulted in file position being lost
if a string was truncated when reading due to the finite buffer size.

fix bug in the gdb reader that could result in data being written outside
the provided buffer.

enhance gdb readers to remove length limit on reading strings.

* whitespace correction.

clion spell fixes, and a few cet retirements. (#370)

* clion spell fixes, and a few cet retirements.

cet retirements were unused.

* more spelling fixes.

resharper fixes (#368)

* fix "redundant statement", "redundant declaration"

* fix newline at eof.

roll Qt to 5.12.4. (#369)

remove dependency on glob. (#367)

* remove dependency on glob.

and have magproto invoke the rd_deinit function when it uses the
gpx reader (uggh).

* regenerate configure on linux.

fix potential memory leaks. (#365)

* fix potential memory leaks.

* fix more memory leaks.

A method to find these is to add -DQT_ASCII_CAST_WARNINGS and
then look for xstrdup in the warnings.

* a few cleanups clazy caught.

Track filter fix (#363)

* Fix compatibility for track filter GUI<->CLI.

Add a week unit for the track filter move option,
in addition to the existing day, hour, minute and second units.

Allow combination of units for track filter move option, e.g.
1w2d3h4m-5s = 1 week + 2 days + 3 hours + 4 minutes - 5 seconds.
This matches historical use, and is expected by the GUI.

Add the week unit to the GUI track filter ui.

Change the limits on the track filter move specs:
week: +/-10000 (unit didn't exit before)
day,hour,minute,second: +/- 100000 (+/-2000 for day before, +/-100 on others).

This resolves #357.
This resolves #358.

* add documentation for track filter move option.

fix package app booboo. (#366)

dmg location was wrong so our attempt to remove any existing
dmg did not work.

enable gui to load Qt provided meta catalogs. (#360)

QTranslate::load fails to load a meta catalog if
the absolute file name is given as the filename parameter, and
the directory parameter is omitted.
It succeeds if the file name is provided as the filename parameter
and the directory is provided as the diretory parameter.

This should resolve #359.

fix stringop-overflow= warning with g++-9. (#362)

Additionally, produce a warning for any conversion errors when
reading XT_TIMET_TIME_MS fields in an xcsv file.

Fix crash on download from Navin/ZNEX miniHomer (#352)

This adds the options gps-utc-offset and gps-week-rollover to the
miniHomer driver. By adding them solely to the skytraq driver they were
initialized with nullptr for the miniHomer ultimately leading to a
crash.

Signed-off-by: Konrad Gräfe <konradgraefe@aol.com>

update Free Software Foundation address. (#354)

* update Free Software Foundation address.

* revert strptime fsf address change.

fix format-overflow= warning with g++-8 in mkshort.cc (#361)

This is Fedora patch 0006-Fix-Wformat-overflow.patch.

correct name of Exec in desktop file. (#353)

The name of the Exec changed in
https://github.com/gpsbabel/gpsbabel/pull/310

Fix gbfputcstr when using non UTF-8 encoding. (#348)

* Fix gbfputcstr when using non UTF-8 encoding.

This restores gbfputcstr from 75d8ec00f^.
That change could result in implicit conversion of
char* data that was specifically encoded as required for
a format to UTF-16, and then explicitly converted from UTF-16
to UTF-8.  While this was inefficient it could also
result in corruption when the data to be written was encoded
in something other than UTF-8.
For example, the Latin1 encoded 'ö' is '\xf6' will
be converted to the UTF-16 replacement character, and that will
be converter to the UTF-8 replacement character, not the original
character.

This resolves a mystery that was noted in the gdb test where some
strings where corrupted when writing gdb versions 1 and 2.

Instead of using function overloading I deleted the version of
gbfputcstr that accepted a QString to write, forcing the user
to specifically convert the data as appropriate.

* increase accuracy of GPS_Math_Deg_To_Semi.

Use rounding of the calculated floating point value before
conversion to an integer to improve accuracy.
This minimizes
abs(GPS_Math_Semi_to_Deg(GPS_Math_Deg_To_Semi(fpvalue)) - fpvalue).
This allows us do some round trip testing from formats that use
a textual representations of degrees, e.g. gpx, to a binary
format that stores integer represetations of angles in semicircle
units, and back.

It is still possible for round trip differences to occur,
but in all our test cases this hasn't been observed.
The value, the number of decimal places printed for the textual
representation, and the size of floating point variables can
all influence this.

Two GPSBabel generated binary reference files are updated.  All changes
in these files are 1 bit changes in latitude or longitude values as
can be expected from the difference between truncation and rounding.

Two GPSBabel generated gpx reference files are deleted as the source
of the round trip test can be compared to the output now.

A gdb test comparison is now enabled as the round trip results now
match.

* enhance gdb test.

adding a roundtrip test with gdb version 3.

Merge pull request #347 from gpsbabel/gpsbabel-patch-3

Create skeletal News

Delete newnews

Update NEWS

Create skeletal News

Add skeletal NEWS file.

Merge pull request #346 from tsteven4/r160

Roll version number for release 1.6.0.

Roll version number for release 1.6.0.

Merge pull request #345 from gpsbabel/gpsbabel-patch-2

Remove outdated multiline comment in gdb.cc

Merge pull request #340 from tsteven4/qt5123

Roll Qt to 5.12.3 and 5.9.8

Remove outdated multiline comment in gdb.cc

hook GUI up to sort filter. (#342)

because the sort filter now works on waypoints
and/or routes and/or tracks the dialog is under
the Miscellaneous menu.

pull minizip from configure flow. (#339)

while it was planned to use minizip those
plans were never completed and it was unused.

zlib/contrib/minizip appears to be unmaintained.
some issues e.g. #338.

Roll Qt to 5.12.3 and 5.9.8

Merge pull request #336 from gpsbabel/datefix

Fix date in Windows rc copyright.

Fix date in Windows rc copyright.

Fix order of arguments to new upgrade check.

Check in the regenerated setup.iss

Turn off debugging mode in upgrade check.

Fix GUI upgrade check to handle upgrade from x-betaFOO to x.

Bump version number for (hopefully short-lived) beta.

Update dependencies in Makefile.in for release.

split csv lines as QStrings and return a list of all fields. (#332)

* introdcue csv_linesplit

Which is like csv_lineparse, except it gives you a list
of all the values, and it processing the line as a QString.

* introduce gpsbabel::TextStream, use it

in unicsv reader, ozi.

* convert unicsv writer to textstream.

* if a codec is not found list available.

* switch unicsv to CET_CHARSET_UTF8 to avoid

undesired fs conversions.  All conversions are
handled by the codec used by qtextstream.

* output boms with non utf8 unicode codecs.

* use rfc4180 dequote method with unicsv.

* add test for csv quoting w RFC4180.

add documentation for some options. (#334)

* add documentation for some options.

garmin_gpi: languagecode
gpx: elevprec

* doc wording tweak.

* delete obsolete unused lowranceusr option,

and document new lowranceusr options.

* tweak doc.

update continous release version for macos. (#331)

update Qt 5.12 builds to 5.12.2. (#329)

* update Qt 5.12 builds to 5.12.2.

* update coverage reporter, kill java8 in docker, add bear.

retire queues. (#328)

 Introduce a WaypointList class to define the interface to our lists of waypoints. (#326)

* Introduce a WaypointList class to define the interface to our lists of waypoints.

WaypointList is backed QList, eliminating the use of legacy queues for waypoint lists.

* try to appease xcode wrt std::abs.

* clean up inheritence in WaypointList, RouteList.

also quiet clazy with igc and gpsbabel_optional::optional.
also improve includes in defs.h.

* get rid of unnecessary reinterpret_cast.

and correct pointer constness lost by auto.

replace queues in mkshort. (#327)

Also, hide the contents of mkshort_handle_imp.

fiddle with nmea waypoint status tracking. (#325)

Traditionally nmea looks at the Waypoint data member Q
to decide if the Waypoint has been added on to a global list, and
if it has not it deletes the Waypoint so we don't leak memory.

This is problematic when we change the waypoint lists so queues
aren't used.

To overcome this difficulty nmea now uses a derived class of
Waypoint that tracks if the Waypoint has been added or not.

fix stackfilter issues. (#324)

1. correct defect in the new RouteList::swap method.
2. correct an old bug in stackfilter that cleaned up
waypts on exit, but not routes or tracks.
3. enhance stackfilter test to test routes and tracks as well
as waypts.

replace random queues with QLists. (#323)

Merge pull request #296 from tsteven4/routelist

Introduce a RouteList class.

convert ozi io to QTextStream (#315)

* convert ozi io to QTextStream.

and add an option to set the codec.
default the codec to windows-1252, which matches historic
usage but not recent behavior.

fix a memory leak, csv_lineparse needs to die.

fix a mistranslation with QString::arg.

* enhance ozi test for routes and tracks.

* eliminate csv_lineparse in ozi.

* eliminate obsolete commented code in ozi.

* update encoding comments for ozi.

* add doc for ozi codec option.

regenerate translation files including command line strings. (#319)

* spelling fixes in mainwindow.cc

* regenerate compiled translation files.

add a process to capture the translated command line strings, &
update the corresponding translation source and compiled files.

remove gui context from gpsbabel_it.ts.

correct errors in mainwindow setting up command line translations.

deploy compiled command line translations.

* correct permissions on new pro file.

* delete obsolete and empty translation files.

add CMakeLists.txt file for GUI. (#322)

Merge pull request #298 from gpsbabel/skytraq

Essentially revert the skytraq part of PR#286.

Merge pull request #321 from gpsbabel/gpsbabel-patch-3

Drop unused get_crc32_s

Merge pull request #320 from gpsbabel/gpsbabel-patch-2

Remove dead code: get_crc32_s

Merge branch 'master' into skytraq

Drop unused get_crc32_s

Remove dead code: get_crc32_s

clean up gui main and fix a png. (#313)

* clean up gui main and fix a png.

- fix libpng warning: iCCP: known incorrect sRGB profile
  issue with runit.png with pngcrush.

- fix includes for gui/main.cc, including removal of
  CoreFoundation.h
- use QApplication static methods instead of going directly
  to the underlying QCoreApplication static methods.

 - no path setting in GUI, instead pass the full path to gpsbabel when using QProcess::start.

in formatload.cc xlt() use qt methods to get a char* instead of
resorting to stdlib.  At this point Qt assumes all translate strings
are utf8.  However, the formatload xlt intent doesn't work because
1) MainWindow::LoadLanguage hasn't been called when
   MainWindow::loadFormats is called.
2) the translations don't exist for the format bits from gpsbabel
   that fromatload tries to translate.  I think this was the intent
   of the gpsbabel_*.ts files which are very out of date and no
   longer distributed.

spelling fixes in mainwindow.cc (#318)

normalize connections in GUI (#316)

as suggested by clazy.

Merge pull request #312 from tsteven4/tidygui3

clang-tidy fixes for the gui

clang-tidy readability-else-after-return on gui.

and reformat fixes with astyle.

manual fix suggested by clang-tidy modernize-use-using.

clang-tidy readability-named-parameter on gui.

clang-tidy hicpp-use-override on gui.

clang-tidy readability-simplify-boolean-expr on gui.

clang-tidy readability-braces-around-statements on gui.

clang-tidy readability-static-accessed-through-instance on gui.

clang-tidy readability-implicit-bool-conversion on gui.

clang-tidy modernize-use-nullptr on gui.

clang-tidy readability-delete-null-pointer on gui.

clang-tidy readability-container-size-empty on gui.

clang-tidy readability-redundant-member-init on gui.

clang-tidy performance-unnecessary-value-param on gui.

clang-tidy hicpp-use-equals-default on gui.

reformat gui with astyle. (#311)